Repository navigation
[0.9.0] Season seats: setAsideSeasonBlock, retrieveSeasonBlock, releaseSeasonBlock, inventoryMode - #13
Merged
Conversation
…h wire tests, events seriesKey
…he p2a-api review
…tle (2.16c) createEventSeries takes fromEventKey: an event already set up becomes the first date under its own key and every other date is a copy of it; name and chartKey are then left out. Every date of a series carries the series' plain name. Matches seatmap slice 2.16c.
Adds releaseSeasonLapsedOffers (POST /v1/seasons/{key}/renewal-offers/release-lapsed, no body, header replay, 202 with the job). retrieveSeasonHostedRenewals pages with cursor and limit (1-500) and answers nextCursor, and listSeasonRenewalOffers pages the same way (1-100). createSeasonRenewalOffers answers 201 {offers} for 100 contracts or fewer and 202 {job, batchId} above that. The bring-across (createSeasonHolderImportFromSeason) answers 202 with a holder_import job above 100 holders. Renewal emails take up to 20,000 offer ids; above that, 422 too_many_offer_ids.
…e_abandoned (p2b) The public API changes in p2b (b88ed794e..ee7df2921): Seasons::dateCancelVoidJobId names the date_cancel_void job a date cancel starts; Seasons::holderImportRowsPerCall sizes an import call by its dates (a call over it answers 422 invalid_rows with maxRows); updateSeason documents singleTicketDates.jobRefused; createEventSeries documents 409 create_abandoned.
…b audit) updateEventSeries sends only the prices that change: each date keeps its own price for every category or tier left out. A season's null transfer or return cut-off means the club default from Settings > Ticket site, or until the date starts when the club has none. Holder transfers and returns through the secret key are not stopped by the club's customer switches; cut-offs still apply.
…part C) A date its season cancelled never opens again: moving its sales to testable or live, or reopening it, answers 409 season_date_cancelled, and an event series "sales on" leaves it closed as a failed item of the job (season_date_cancelled; not in failedDates). A cancelled date can't be moved, postponed, replaced or marked as taken place: 409 performance_cancelled.
pietechsolution
force-pushed
the
navin/season-block-0.10.0
branch
2 times, most recently
from
November 10, 2026 02:32
5c50999 to
8cf1533
Compare
…seSeasonBlock; Seasons::isDateLeavePending for the 202 date-leave answer. No tag: publish only once the phase 4 API is on production
pietechsolution
force-pushed
the
navin/season-block-0.10.0
branch
from
November 10, 2026 05:48
8cf1533 to
cc655a8
Compare
Merged
pietechsolution
marked this pull request as ready for review
November 11, 2026 10:14
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Season seats for
seatlayer/seatlayer-php, part of the 0.9.0 release. One release, 0.9.0, carries season 2a (#12) and this PR (decided 10 Nov).v0.8.0(checked 10 Nov).season-2a-serverso it builds on the season-2a methods. Its name still says 0.10.0, from before the decision: renaming a head branch would close the PR.composer.jsonhas no version, by design: on Packagist a tag is the publish.What it adds
Built to
docs/season-block-2026-11-08/contract.mdin the app repo (§2, §5, §6, and §8 "Amendments, 9 Nov"). Method names are the operationIds.Methods:
$seasons->setAsideSeasonBlock($seasonKey, ['expectedRevision' => …, 'keepForSingleTickets' => [...]], $idempotencyKey);$seasons->retrieveSeasonBlock($seasonKey);$seasons->releaseSeasonBlock($seasonKey, ['expectedRevision' => …, 'scope' => [...], 'eventKeys' => [...], 'at' => …, 'reason' => …], $idempotencyKey).Both writes use header replay, so an Idempotency-Key is generated when none is given, as on
refundSeasonPerformance. Each write answersjobandblock; follow the job withretrieveSeasonBulkJob(). The array shapes are typed for PHPStan.inventoryMode(block|per_hold) oncreateSeason()andupdateSeason(), in the typed array shapes and sent only when given.blockopts the season into season seats; the default staysper_hold. Every Season answer carries it.The 202 while a date leaves every season seat (
createSeasonAmendmentcancel,markSeasonPerformancePast):Seasons::isDateLeavePending($answer). The return types are unchanged.Webhooks:
Webhookssends any event name, soseason.block.readyandseason.block.releasedneed no change. The CHANGELOG names them.Errors: no new classes or constants. The new codes come through
SeatLayerExceptionand its subclasses witherrorCode(§8 item 4). The CHANGELOG lists them, now including every code the 3 calls andinventoryModecan send (SEASON_BLOCK_API_BUSINESS_ERRORSin the app repo).Docs: a "Season seats" section at the end of the README's "Fixed Renewable Seasons": opt in with
inventoryMode, set the seats aside, follow the job, read the block, release to single sale (now or at a set time), the two webhook names, the 202 guard, refusals throughSeatLayerExceptionwitherrorCode, and partner seats staying out of the block.CHANGELOG: the "Next: 0.9.0" section. The version PR joins it with the season-2a section under
## 0.9.0.Files changed
CHANGELOG.md,README.md,src/Resources/Seasons.php,tests/SeasonTest.php(4 files, +266 −9).Checks
The repo's gate, re-run 10 Nov on this commit:
vendor/bin/phpstan analyse(level 8): no errors;vendor/bin/phpunit: 77/77, 468 assertions.This ran on PHP 8.5 locally (CI runs 8.1–8.4). PHPStan needs
--memory-limit=1Ghere; that is this machine's php.ini, not the code.ConflictExceptionwith itserrorCode.inventoryModeon create and update.Mutation check (run before this amend; the code is unchanged since): 7 mutants (the 3 paths, header replay, the caller's key, the guard's two halves): 7/7 killed.
The README sample isn't run by the repo. Its syntax was checked once (
php -l), and the calls by hand against the signatures and PHPStan shapes inSeasons.php.Compat matrix: the same fixtures, shaped as the phase 4 API answers, through a stub transport. The script is
season-phase4-php-compat.phpin the app session's notes; nothing is sent.composer require seatlayer/seatlayer-php:0.8.0, re-run 10 Nov)createSeasonAmendment, a Season answer carryinginventoryMode, and409 season_block_packagesonupdateSeason. The other 10 are n/a: 0.8.0 has neither the season-2a calls nor theseseason-2a-serverhead62bf35dinventoryModeand409 season_block_packagessrc/), including the 3 new calls and409 season_block_not_enabledRelease steps for the owner
The season-2a PR (Season 2a server operations (part A) #12) merges to
main, with no tag.Rebase this PR onto
mainand retarget it tomain; the gate green, andcomposer validate --strict; merge.Compat matrix again, once the phase 4 API is on dev:
main'ssrc/: both all PASS;completed, read, release withat, read the release job.Version PR: there is no version field. Join the CHANGELOG's season-2a section ("Unreleased") and "Next: 0.9.0" under a dated
## 0.9.0. Merge tomain.Publish: only after the phase 4 API (paiteq/seatmap #2059 → #2086 → #2061 → #2062 → #2063 → #2066 → #2092) is live on production and the owner says yes: tag the merge commit,
git tag v0.9.0 && git push origin v0.9.0. Checkgit rev-list -n1 v0.9.0equalsgit rev-parse mainfirst (RELEASE.md§2.0): a tag on the wrong commit publishes the wrong tree.Live re-check:
v0.9.0;composer require seatlayer/seatlayer-php:0.9.0, then run the dev smoke from it.Rollback (0.9.0 only adds). Packagist has no unlist (
RELEASE.md§5): it mirrors the tags.v0.9.0, ask users to pin0.8.0, and tag the fix asv0.9.1.v0.9.0tag on GitHub, then delete the version on Packagist. After real installs this breaks anyone who pinned it.Either choice covers season-2a too: both ship in 0.9.0.
Failure modes
inventoryModewould be ignored. Step 5 waits for production.Seasons::isDateLeavePending(): it would read the 202 body as the amendment. The README and the doc comments point at the guard.inventoryMode(compat matrix). It can't call the 3 new routes.Open risks
/server-api/seasons/#season-seats), which goes live with paiteq/seatmap #2063. Until then the link lands on the top of the Seasons page.